Skip to content

fix: remove no longer relevant bootstrap-cli command from init task#261

Merged
Boshen merged 1 commit intovoidzero-dev:mainfrom
yangshun:patch-1
Mar 18, 2026
Merged

fix: remove no longer relevant bootstrap-cli command from init task#261
Boshen merged 1 commit intovoidzero-dev:mainfrom
yangshun:patch-1

Conversation

@yangshun
Copy link
Contributor

Description

Was trying to set up the repo by following the instructions in CONTRIBUTING.md, but encountered an error

➜  vite-task git:(main) just init
cargo binstall watchexec-cli cargo-insta typos-cli cargo-shear@1.11.1 taplo-cli -y
 INFO resolve: Resolving package: 'typos-cli'
 # Omitted
  ...
    Finished [`release` profile [optimized]](https://doc.rust-lang.org/cargo/reference/profiles.html#default-profiles) target(s) in 47.13s
  Installing /Users/yangshun/.cargo/bin/taplo
   Installed package `taplo-cli v0.10.0` (executable `taplo`)
 INFO Cargo finished successfully
 INFO Done in 53.617422084s
pnpm run bootstrap-cli
 ERR_PNPM_NO_SCRIPT  Missing script: bootstrap-cli

Issue

justfile references pnpm run bootstrap-cli under init, but there's no longer such a script in the package.json.

  1. This line was originally added to the root package.json in f082903
  2. The root package.json was removed in dda69c7, but the setup instructions in justinit weren't updated.

The new package.json doesn't contain the script bootstrap-cli.

Test plan

  1. Removed that line from justfile
  2. Ran just init
  3. Command completed successfully

Removed pnpm run bootstrap-cli from init task.

Signed-off-by: Yangshun Tay <tay.yang.shun@gmail.com>
@yangshun yangshun changed the title fix: remove bootstrap-cli command from init task fix: remove no longer relevant bootstrap-cli command from init task Mar 18, 2026
@Boshen Boshen requested review from Brooooooklyn and removed request for Brooooooklyn March 18, 2026 04:01
@Boshen Boshen merged commit 89bddcd into voidzero-dev:main Mar 18, 2026
7 checks passed
@yangshun yangshun deleted the patch-1 branch March 18, 2026 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants